Skip to content

Added functionalities#6

Open
subdesign wants to merge 6 commits into
danmontgomery:masterfrom
subdesign:master
Open

Added functionalities#6
subdesign wants to merge 6 commits into
danmontgomery:masterfrom
subdesign:master

Conversation

@subdesign

Copy link
Copy Markdown

I added the following functions:

  • optional email activation
  • user registration with reCaptcha
  • forgot password
  • alert message email to admin if user is locked out (optional)

@danmontgomery

Copy link
Copy Markdown
Owner

I dig this. Need to find time in the next few days to look it over, and I'll merge it in.

@subdesign

Copy link
Copy Markdown
Author

All right, feel free to modify or prettify the code if needed.

@CroNiX

CroNiX commented May 26, 2012

Copy link
Copy Markdown

I'm trying to test this but getting the "To use reCAPTCHA you must get an API key from https://www.google.com/recaptcha/admin/create" message. I do have a key entered, but I'm testing this on localhost and google states any key should work on localhost.

@subdesign

Copy link
Copy Markdown
Author

I don't clearly understand the problem with reCaptcha, can you explain in a bit more detail?

@CroNiX

CroNiX commented May 29, 2012

Copy link
Copy Markdown

Call to undefined method Bitauth::get_user_by_email()
Called by Bitauth::generate_forgot_code()

@CroNiX

CroNiX commented May 29, 2012

Copy link
Copy Markdown

@subdesign I was just testing the base install with your mod, set the recaptcha keys in the config (even though I'm testing on localhost/127.0.0.1 and google says any key will work on localhost), and got the error that I reported above. I've disabled it as I don't need it for what I'm doing, but it didn't work for my tests.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this <script> here, I think this should be moved out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants